Open Bug 1725850 Opened 4 years ago Updated 2 years ago

Move crashtests under editor/ to editing/crashtests

Categories

(Core :: DOM: Editor, task, P3)

Product:
Core
Component:
DOM: Editor
Type:
task

Tracking

()

Status:
ASSIGNED

People

(Reporter: masayuki, Assigned: masayuki)

References

(Blocks 1 open bug,
URL
)

Details

(Keywords: leave-open)

Attachments

(35 files)

Bug 1725850 - part 1: Port editor/libeditor/crashtests/336104.html to WPT r=saschanaz!
48 bytes, text/x-phabricator-request
Details | Review
Bug 1725850 - part 2: Port editor/libeditor/crashtests/382527-1.html to WPT r=saschanaz!
48 bytes, text/x-phabricator-request
Details | Review
Bug 1725850 - part 3: Port editor/libeditor/crashtests/382778-1.html to WPT r=saschanaz!
48 bytes, text/x-phabricator-request
Details | Review
Bug 1725850 - part 4: Port editor/libeditor/crashtests/402172-1.html to WPT r=saschanaz!
48 bytes, text/x-phabricator-request
Details | Review
Bug 1725850 - part 5: Port editor/libeditor/crashtests/407074-1.html to WPT r=saschanaz!
48 bytes, text/x-phabricator-request
Details | Review
Bug 1725850 - part 6: Port editor/libeditor/crashtests/407079-1.html to WPT r=saschanaz!
48 bytes, text/x-phabricator-request
Details | Review
Bug 1725850 - part 7: Port editor/libeditor/crashtests/407256-1.html to WPT r=saschanaz!
48 bytes, text/x-phabricator-request
Details | Review
Bug 1725850 - part 8: Port editor/libeditor/crashtests/407277-1.html to WPT r=saschanaz!
48 bytes, text/x-phabricator-request
Details | Review
Bug 1725850 - part 9: Port editor/libeditor/crashtests/414178-1.html to WPT r=saschanaz!
48 bytes, text/x-phabricator-request
Details | Review
Bug 1725850 - part 10: Port editor/libeditor/crashtests/418923-1.html to WPT r=saschanaz!
48 bytes, text/x-phabricator-request
Details | Review
Bug 1725850 - part 11: Port editor/libeditor/crashtests/420439.html to WPT r=saschanaz!
48 bytes, text/x-phabricator-request
Details | Review
Bug 1725850 - part 12: Port editor/libeditor/crashtests/430624-1.html to WPT r=saschanaz!
48 bytes, text/x-phabricator-request
Details | Review
Bug 1725850 - part 13: Port editor/libeditor/crashtests/448329-1.html to WPT r=saschanaz!
48 bytes, text/x-phabricator-request
Details | Review
Bug 1725850 - part 14: Port editor/libeditor/crashtests/448329-2.html to WPT r=saschanaz!
48 bytes, text/x-phabricator-request
Details | Review
Bug 1725850 - part 15: Port editor/libeditor/crashtests/448329-3.html to WPT r=saschanaz!
48 bytes, text/x-phabricator-request
Details | Review
Bug 1725850 - part 16: Port editor/libeditor/crashtests/456727-1.html to WPT r=saschanaz!
48 bytes, text/x-phabricator-request
Details | Review
Bug 1725850 - part 17: Port editor/libeditor/crashtests/456727-2.html to WPT r=saschanaz!
48 bytes, text/x-phabricator-request
Details | Review
Bug 1725850 - part 18: Port editor/libeditor/crashtests/467647-1.html to WPT r=saschanaz!
48 bytes, text/x-phabricator-request
Details | Review
Bug 1725850 - part 19: Port editor/libeditor/crashtests/459613.html to WPT r=saschanaz!
48 bytes, text/x-phabricator-request
Details | Review
Bug 1725850 - part 20: Port editor/libeditor/crashtests/499044-1.html to WPT r=saschanaz!
48 bytes, text/x-phabricator-request
Details | Review
Bug 1725850 - part 21: Port editor/libeditor/crashtests/612565-1.html to WPT r=saschanaz!
48 bytes, text/x-phabricator-request
Details | Review
Bug 1725850 - part 22: Port editor/libeditor/crashtests/615015-1.html to WPT r=saschanaz!
48 bytes, text/x-phabricator-request
Details | Review
Bug 1725850 - part 23: Port editor/libeditor/crashtests/615450-1.html to WPT r=saschanaz!
48 bytes, text/x-phabricator-request
Details | Review
Bug 1725850 - part 24: Port editor/libeditor/crashtests/636074-1.html to WPT r=saschanaz!
48 bytes, text/x-phabricator-request
Details | Review
Bug 1725850 - part 25: Port editor/libeditor/crashtests/643706-1.html to WPT r=saschanaz!
48 bytes, text/x-phabricator-request
Details | Review
Bug 1725850 - part 26: Port editor/libeditor/crashtests/650572-1.html to WPT r=saschanaz!
48 bytes, text/x-phabricator-request
Details | Review
Bug 1725850 - part 27: Port editor/libeditor/crashtests/667321-1.html to WPT r=saschanaz!
48 bytes, text/x-phabricator-request
Details | Review
Bug 1725850 - part 28: Port editor/libeditor/crashtests/682650-1.html to WPT r=saschanaz!
48 bytes, text/x-phabricator-request
Details | Review
Bug 1725850 - part 29: Port editor/libeditor/crashtests/713427-1.html to WPT r=saschanaz!
48 bytes, text/x-phabricator-request
Details | Review
Bug 1725850 - part 30: Port editor/libeditor/crashtests/716456-1.html to WPT r=saschanaz!
48 bytes, text/x-phabricator-request
Details | Review
Bug 1725850 - part 31: Port editor/libeditor/crashtests/745494.html to WPT r=saschanaz!
48 bytes, text/x-phabricator-request
Details | Review
Bug 1725850 - part 32: Port editor/libeditor/crashtests/759748.html to WPT r=saschanaz!
48 bytes, text/x-phabricator-request
Details | Review
Bug 1725850 - part 33: Port editor/libeditor/crashtests/761861.html to WPT r=saschanaz!
48 bytes, text/x-phabricator-request
Details | Review
Bug 1725850 - part 34: Port editor/libeditor/crashtests/762103.html to WPT r=saschanaz!
48 bytes, text/x-phabricator-request
Details | Review
Bug 1725850 - part 35: Port editor/libeditor/crashtests/766305.html to WPT r=saschanaz!
48 bytes, text/x-phabricator-request
Details | Review
No description provided.
Severity: -- → N/A
Priority: -- → P3
Assignee: nobody → masayuki
Status: NEW → ASSIGNED
Keywords: leave-open

The test used CRLF. Therefore, you may see all lines are changed.

Depends on D124846

Pushed by masayuki@d-toybox.com: https://hg.mozilla.org/integration/autoland/rev/232802020e08 part 1: Port editor/libeditor/crashtests/336104.html to WPT r=saschanaz https://hg.mozilla.org/integration/autoland/rev/3da11343fbf2 part 2: Port editor/libeditor/crashtests/382527-1.html to WPT r=saschanaz https://hg.mozilla.org/integration/autoland/rev/d97f699612eb part 3: Port editor/libeditor/crashtests/382778-1.html to WPT r=saschanaz https://hg.mozilla.org/integration/autoland/rev/f744eb441242 part 4: Port editor/libeditor/crashtests/402172-1.html to WPT r=saschanaz https://hg.mozilla.org/integration/autoland/rev/b7e917f0c9cc part 5: Port editor/libeditor/crashtests/407074-1.html to WPT r=saschanaz https://hg.mozilla.org/integration/autoland/rev/105b4e292d99 part 6: Port editor/libeditor/crashtests/407079-1.html to WPT r=saschanaz https://hg.mozilla.org/integration/autoland/rev/dcd667a9d8e6 part 7: Port editor/libeditor/crashtests/407256-1.html to WPT r=saschanaz https://hg.mozilla.org/integration/autoland/rev/0b7cd5022143 part 8: Port editor/libeditor/crashtests/407277-1.html to WPT r=saschanaz https://hg.mozilla.org/integration/autoland/rev/1cbe007d5c33 part 9: Port editor/libeditor/crashtests/414178-1.html to WPT r=saschanaz https://hg.mozilla.org/integration/autoland/rev/044710dfd5d1 part 10: Port editor/libeditor/crashtests/418923-1.html to WPT r=saschanaz
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/30545 for changes under testing/web-platform/tests
Upstream PR merged by moz-wptsync-bot

The test requires to support allowing nested execCommand if we want to run
completely. From point of view of the purose of crashtests, we should allow
it for testing in more complicated situation. Therefore, this patch allows
it while the test is running by the pref.

I think that the crashtests marked as needs-focus should wait focus event
in the document. I think that focus event is fired only when the window has
focus (*1) and Element.focus() steals focus unless chrome UI has focus (*2).
If the testing document hasn't get focus yet, focus event will be dispatched
later (*3).

1: https://searchfox.org/mozilla-central/rev/d8d7539593d5a3071841289dee72f971650073ce/dom/base/nsFocusManager.cpp#1672-1681
2: https://searchfox.org/mozilla-central/rev/d8d7539593d5a3071841289dee72f971650073ce/dom/base/nsFocusManager.cpp#1683-1691
3: https://searchfox.org/mozilla-central/rev/d8d7539593d5a3071841289dee72f971650073ce/dom/base/nsFocusManager.cpp#759-777

Depends on D125920

Pushed by masayuki@d-toybox.com: https://hg.mozilla.org/integration/autoland/rev/4b96fdb4c8f7 part 11: Port editor/libeditor/crashtests/420439.html to WPT r=saschanaz https://hg.mozilla.org/integration/autoland/rev/48943ed3ffd0 part 12: Port editor/libeditor/crashtests/430624-1.html to WPT r=saschanaz https://hg.mozilla.org/integration/autoland/rev/68f6d9d84d82 part 13: Port editor/libeditor/crashtests/448329-1.html to WPT r=saschanaz https://hg.mozilla.org/integration/autoland/rev/b14b5f1f3a98 part 14: Port editor/libeditor/crashtests/448329-2.html to WPT r=saschanaz https://hg.mozilla.org/integration/autoland/rev/5e31e98f09af part 15: Port editor/libeditor/crashtests/448329-3.html to WPT r=saschanaz https://hg.mozilla.org/integration/autoland/rev/8178c73014a9 part 16: Port editor/libeditor/crashtests/456727-1.html to WPT r=saschanaz https://hg.mozilla.org/integration/autoland/rev/1c196e74bfc9 part 17: Port editor/libeditor/crashtests/456727-2.html to WPT r=saschanaz https://hg.mozilla.org/integration/autoland/rev/9de37f824fef part 18: Port editor/libeditor/crashtests/467647-1.html to WPT r=saschanaz
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/31009 for changes under testing/web-platform/tests
Upstream PR merged by moz-wptsync-bot
Pushed by masayuki@d-toybox.com: https://hg.mozilla.org/integration/autoland/rev/7765e08df4fa part 19: Port editor/libeditor/crashtests/459613.html to WPT r=saschanaz https://hg.mozilla.org/integration/autoland/rev/b52d6f61c208 part 20: Port editor/libeditor/crashtests/499044-1.html to WPT r=saschanaz https://hg.mozilla.org/integration/autoland/rev/8fa7d9e2a4c9 part 21: Port editor/libeditor/crashtests/612565-1.html to WPT r=saschanaz https://hg.mozilla.org/integration/autoland/rev/b849c1dd7585 part 22: Port editor/libeditor/crashtests/615015-1.html to WPT r=saschanaz https://hg.mozilla.org/integration/autoland/rev/057de5e308fe part 23: Port editor/libeditor/crashtests/615450-1.html to WPT r=saschanaz https://hg.mozilla.org/integration/autoland/rev/16d0c8cdc167 part 24: Port editor/libeditor/crashtests/636074-1.html to WPT r=saschanaz
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/31129 for changes under testing/web-platform/tests
Upstream PR merged by moz-wptsync-bot
Pushed by masayuki@d-toybox.com: https://hg.mozilla.org/integration/autoland/rev/97e71464e24e part 25: Port editor/libeditor/crashtests/643706-1.html to WPT r=saschanaz https://hg.mozilla.org/integration/autoland/rev/f23111778411 part 26: Port editor/libeditor/crashtests/650572-1.html to WPT r=saschanaz https://hg.mozilla.org/integration/autoland/rev/113669447639 part 27: Port editor/libeditor/crashtests/667321-1.html to WPT r=saschanaz https://hg.mozilla.org/integration/autoland/rev/6e9d68ba9c21 part 28: Port editor/libeditor/crashtests/682650-1.html to WPT r=saschanaz https://hg.mozilla.org/integration/autoland/rev/d209f3ee4d5b part 29: Port editor/libeditor/crashtests/713427-1.html to WPT r=saschanaz https://hg.mozilla.org/integration/autoland/rev/c974b2b8004c part 30: Port editor/libeditor/crashtests/716456-1.html to WPT r=saschanaz
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/31435 for changes under testing/web-platform/tests
Upstream PR merged by moz-wptsync-bot
Pushed by masayuki@d-toybox.com: https://hg.mozilla.org/integration/autoland/rev/3cd7cabd4459 part 31: Port editor/libeditor/crashtests/745494.html to WPT r=saschanaz https://hg.mozilla.org/integration/autoland/rev/beba43f4ce5b part 32: Port editor/libeditor/crashtests/759748.html to WPT r=saschanaz https://hg.mozilla.org/integration/autoland/rev/a0069e962af5 part 33: Port editor/libeditor/crashtests/761861.html to WPT r=saschanaz https://hg.mozilla.org/integration/autoland/rev/8e47246b16bd part 34: Port editor/libeditor/crashtests/762103.html to WPT r=saschanaz https://hg.mozilla.org/integration/autoland/rev/428f37efeef6 part 35: Port editor/libeditor/crashtests/766305.html to WPT r=saschanaz
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/33260 for changes under testing/web-platform/tests
Upstream PR merged by moz-wptsync-bot

The leave-open keyword is there and there is no activity for 6 months.
:hsivonen, maybe it's time to close this bug?
For more information, please visit auto_nag documentation.

Flags: needinfo?(hsivonen)

No, we should keep moving them to WPT at least for *.html.

Flags: needinfo?(hsivonen)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: